chore(ci): test summary step - #20
Conversation
Adds a Summary job (mirrors cap-js/mcp pattern) that needs [test, test-hybrid] and exits 1 on any failure or cancellation. gh_ruleset.json makes Summary the single required check on main, replacing individual matrix job entries.
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
…ntain permissions' Co-authored-by: Copilot Autofix powered by AI <62310815+github-advanced-security[bot]@users.noreply.github.com>
|
Required checks have to be adapted in GH repo settings, will do after review |
SummaryThe following content is AI-generated and provides a summary of the pull request: chore(ci): Optimize CI test matrix and add summary stepCategory: Chore This PR refines the CI workflow to reduce unnecessary test runs on pull requests while ensuring full coverage in the merge queue. Key changes to
Have you...
PR Bot InformationVersion:
💌 Have ideas or want to contribute? Create an issue and share your thoughts with us! Made with ❤️ by Hyperspace. |
| if: always() | ||
| runs-on: ubuntu-latest | ||
| name: Summary | ||
| needs: [test, test-hybrid] |
There was a problem hiding this comment.
is the hybrid test result from the PR checks reused in the merge queue or does it need to run again?
There was a problem hiding this comment.
In this case it's run again. I'd prefer to move hybrid checks to merge queue only if you're okay with not having the PR checks.
|
Draft until hybrid tests are < 10 min |
3 participants
Similar to cap-js/mcp#35, to delay the extended test suite to the merge queue.
Just running Node 24 + latest cds in PRs, as that's triggered for every push.
Have you...